🌱 E2E: Use kind cluster for clusterctl upgrade tests #2792
Conversation
k8s-ci-robot
added
the
do-not-merge/hold
k8s-ci-robot
added
the
size/XS
|
/test pull-cluster-api-provider-openstack-e2e-full-test |
✅ Deploy Preview for kubernetes-sigs-cluster-api-openstack ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This enables the UseKindForManagementCluster flag for all clusterctl upgrade tests, which makes them use a kind cluster as the secondary management cluster instead of creating an OpenStack workload cluster. This significantly reduces the resource requirements for these tests by using lightweight kind containers instead of full VMs, and it also speeds up the test execution. Signed-off-by: Lennart Jern <[email protected]>
lentzi90
force-pushed
the
lentzi90/docker-cluster
branch
from
61c9385 to
e7c1b8d
Compare
|
/test pull-cluster-api-provider-openstack-e2e-full-test |
|
Looks like it works, and saves a few minutes for the test also! |
k8s-ci-robot
removed
the
do-not-merge/hold
|
There is a bit of cleanup we could do after this also, but let's take it separately in case we want to revert for some reason.
These are used to upload the CAPO image that is being tested into the management cluster. All other tests use kind clusters where the image is loaded directly, but the clusterctl upgrade was using a full CAPO cluster so we used this to get the image into it. |
|
Unknown CLA label state. Rechecking for CLA labels. Send feedback to sig-contributor-experience at kubernetes/community. /check-cla |
1 similar comment
|
Unknown CLA label state. Rechecking for CLA labels. Send feedback to sig-contributor-experience at kubernetes/community. /check-cla |
k8s-ci-robot
added
the
cncf-cla: yes
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lentzi90 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
k8s-ci-robot
added
the
approved
k8s-ci-robot
added
the
lgtm
4 participants
What this PR does / why we need it:
This enables the UseKindForManagementCluster flag for all clusterctl upgrade tests, which makes them use a kind cluster as the secondary management cluster instead of creating an OpenStack workload cluster.
This significantly reduces the resource requirements for these tests by using lightweight kind containers instead of full VMs, and it also speeds up the test execution.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close the issue(s) when PR gets merged):Fixes #2098
Special notes for your reviewer:
TODOs:
/hold